🤖 Copilot Code Review — PR #128377

Holistic Assessment

Motivation: This is a CI validation PR for install script changes from dotnet/install-scripts. The PR description clearly states it should not be merged and will be closed once CI passes. This is a standard workflow for validating install script updates before they are promoted.

Approach: The change temporarily redirects the install script download URLs from the official builds.dotnet.microsoft.com endpoint to a raw GitHub content URL pointing at a specific commit (9c8552ae791982d3674fbb4d6ad887e536e7f506). This is the expected pattern for validation PRs.

Summary: ✅ LGTM. This is a standard install-scripts validation PR. The changes are minimal, scoped correctly, and clearly marked as DO NOT MERGE. Both tools.ps1 and tools.sh are updated consistently to the same commit hash.

Detailed Findings

✅ Consistency — Both scripts point to the same commit

Both eng/common/tools.ps1 and eng/common/tools.sh reference the same commit SHA (9c8552ae791982d3674fbb4d6ad887e536e7f506), ensuring consistent behavior across Windows and Linux CI.

✅ Scope — Changes are minimal and focused

Only the URL strings are changed (one line each in two files). No other modifications are present.

💡 Reminder — Do not merge

As stated in the PR description, this should be closed after CI validates successfully. The eng/common directory is synced from dotnet/arcade and should not carry manual modifications long-term.

Generated by Code Review for issue #128377 · ● 1.5M · ◷